The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 3, 2026 10:56pm

PR Summary

Low Risk
Low risk UI-only change to the integrations members table; main concern is minor permission/disable-state regressions for member role editing/removal.

Overview
Readonly users now see the same (disabled) role Combobox in the integrations members table instead of a separate badge-style role display.

The role selector JSX is deduplicated into a single Combobox with conditional disabled logic, and roleComboOptions is hoisted to module scope to avoid per-render allocations.

^{Reviewed by Cursor Bugbot for commit 768eda9. Configure here.}

Greptile Summary

This PR fixes a visual inconsistency in the integrations members table where readonly (non-admin) users saw an unstyled Badge for their role instead of the same disabled Combobox that sole-admin members saw. The fix collapses two conditional Combobox branches into one, controlled by a unified disabled prop (!isSelectedAdmin || (member.role === 'admin' && adminMemberCount <= 1)), and lifts the roleComboOptions mapping to module scope to avoid per-render object allocation.

• Bug fixed: Readonly members now see a consistent, disabled Combobox matching the admin-row UI.
• Refactor: Duplicate Combobox JSX collapsed into a single instance; roleComboOptions extracted to module level.
• Layout preserved: The 3-column grid [1fr_120px_72px] for member rows is unaffected — the third column still renders either a Remove button (for admins) or an empty <div /> (for non-admins).
• No logic regressions: The onChange handler is always wired but the Combobox is disabled for non-admins, so handleChangeMemberRole cannot be triggered from the UI in that case.

Confidence Score: 5/5

Safe to merge — the change is a targeted UI fix with no risk of data mutation for readonly users.

All findings are P2 or lower. The core logic is correct: the disabled expression exactly mirrors the previous per-branch logic, the grid layout is preserved, and onChange cannot fire on a disabled Combobox. No tests were added but the change is purely presentational.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Render member row] --> B{isSelectedAdmin?}
    B -- Yes --> C[Combobox: disabled if sole admin]
    B -- No --> D[Combobox: always disabled]
    C --> E{isSelectedAdmin?}
    D --> E
    E -- Yes --> F[Remove Button]
    E -- No --> G[empty div]

_{Reviews (1): Last reviewed commit: "fix(integrations): show disabled role co..." | Re-trigger Greptile}